Skip to content

gordon: clarify Business tier activation via Settings Management#25153

Merged
dvdksn merged 2 commits into
docker:mainfrom
dvdksn:gordon-business-settings-mgmt
May 27, 2026
Merged

gordon: clarify Business tier activation via Settings Management#25153
dvdksn merged 2 commits into
docker:mainfrom
dvdksn:gordon-business-settings-mgmt

Conversation

@dvdksn
Copy link
Copy Markdown
Contributor

@dvdksn dvdksn commented May 22, 2026

Summary

The Business tier activation note in the Gordon prerequisites linked directly to the Admin Console configuration page but didn't signal that Settings Management as a whole needs to be properly set up for the policy to reach Docker Desktop clients. Updated step 2 to link to the top-level Settings Management page and note that its prerequisites must be met — deferring to that page as the canonical source rather than duplicating the details here.

Generated by Claude Code

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented May 22, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 0137a32
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6a16ffedf3d42200080acea2
😎 Deploy Preview https://deploy-preview-25153--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@dvdksn dvdksn added the status/review Pull requests that are ready for review label May 22, 2026
@dvdksn dvdksn requested a review from a team May 22, 2026 12:13
Copy link
Copy Markdown

@docker-agent docker-agent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assessment: 🟢 APPROVE

The change is focused and accurate — it correctly redirects users to the top-level Settings Management page and adds a helpful prerequisite note. No high or medium issues found.

Comment thread content/manuals/ai/gordon/_index.md Outdated
Comment thread content/manuals/ai/gordon/_index.md
- Update settings-management link to use _index.md extension so htmltest
  resolves it (matches repo convention)
- Replace 'enable Gordon' with 'turn on Gordon' per Docker word list
- Reframe prerequisite note positively per style guide
@dvdksn dvdksn merged commit 70629ac into docker:main May 27, 2026
18 checks passed
@dvdksn dvdksn deleted the gordon-business-settings-mgmt branch May 27, 2026 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ai status/review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants